7.25 /review - 请求代码审查

概述#

/review 命令用于请求代码审查，让 Claude 帮助你检查代码质量、发现潜在问题并提供改进建议。

基本语法#

bash
复制
/review [file-or-code]

参数说明#

审查内容#

执行 /review 命令后，Claude 会检查：

1. 代码质量

   • 代码风格一致性
   • 命名规范
   • 代码结构

2. 潜在问题

   • 逻辑错误
   • 边界情况
   • 性能问题

3. 安全性

   • 安全漏洞
   • 输入验证
   • 数据保护

4. 最佳实践

   • 设计模式
   • 代码复用
   • 可维护性

使用场景#

1. 审查文件#

bash
复制
# 审查特定文件
/review src/utils.js

# 检查文件中的代码

2. 审查代码片段#

bash
复制
# 审查代码片段
/review

# 然后粘贴代码片段

3. 审查当前更改#

bash
复制
# 审查当前更改的代码
/review

# Claude 会审查最近的修改

审查报告示例#

bash
复制
代码审查报告
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

文件: src/utils.js

✓ 优点:
- 函数命名清晰，易于理解
- 错误处理完善
- 代码注释详细

⚠ 建议:
1. 第 15 行：可以使用更简洁的语法
   当前: if (condition) { return true; } else { return false; }
   建议: return condition;

2. 第 28 行：建议添加输入验证
   当前: function process(data) { ... }
   建议: function process(data) {
     if (!data) throw new Error('Invalid data');
     ...
   }

3. 第 42 行：考虑使用 const 替代 let
   当前: let result = calculate();
   建议: const result = calculate();

✗ 问题:
1. 第 56 行：潜在的内存泄漏
   问题: 未关闭文件句柄
   修复: 添加 file.close()

总体评分: 7/10

审查选项#

指定审查重点#

bash
复制
# 专注于安全性
/review --focus security

# 专注于性能
/review --focus performance

# 专注于代码风格
/review --focus style

排除某些检查#

bash
复制
# 排除性能检查
/review --exclude performance

# 排除风格检查
/review --exclude style

注意事项#

1. 审查结果基于静态分析
2. 建议需要人工验证
3. 审查不会修改代码
4. 可以多次审查同一代码

与其他命令的配合使用#

bash
复制
# 审查代码，然后修复
/review
# 根据建议修改代码

最佳实践#

1. 定期审查：定期审查代码，保持代码质量
2. 重点审查：在提交前审查代码
3. 团队协作：与团队成员一起审查代码
4. 持续改进：根据审查建议持续改进代码

总结#

/review 命令是 Claude Code 中用于代码审查的重要命令。通过使用此命令，你可以：

• 检查代码质量
• 发现潜在问题
• 获取改进建议
• 提高代码质量

合理使用 /review 命令可以帮助你保持代码的高质量。